Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: update container runtime archive to nerdctl 2.0.2 #472

Conversation

austinvazquez
Copy link
Member

@austinvazquez austinvazquez commented Dec 25, 2024

Issue #, if available:
#475

Description of changes:
This change updates the container runtime full archive used in Finch on macOS and Windows to nerdctl v2.0.2. See https://github.com/containerd/nerdctl/releases/tag/v2.0.2 for full details.

The archive was manually generated and includes a patch for containerd/nerdctl@aa5eb28 which was accepted and merged upstream but is not yet available in a tagged release.

Testing done:
Ran make $(pwd)/_output/lima-template/fedora.yaml and verified lima configuration was correct.

  • I've reviewed the guidance in CONTRIBUTING.md

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

.github/workflows/update-dependencies.yaml Outdated Show resolved Hide resolved
e2e/go.mod Outdated Show resolved Hide resolved
This change updates the container runtime full archive to nerdctl
v2.0.2-1735857497 which was manually built from nerdctl v2.0.2 and
cherry-picks containerd/nerdctl@f917e5c

Signed-off-by: Austin Vazquez <[email protected]>
@austinvazquez austinvazquez force-pushed the feat-container-runtime-full-archive branch from 4b4812d to e4256fc Compare January 2, 2025 23:09
@austinvazquez
Copy link
Member Author

Going to bring this in and open Finch PR for integration testing. Happy to follow-up further if required.

@austinvazquez austinvazquez merged commit 49ebd39 into runfinch:main Jan 3, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants